Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uses stripe.handleNextAction for payment intents #908

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

chrissrogers
Copy link
Member

@chrissrogers chrissrogers commented Nov 21, 2024

For some payment methods handle their PaymentIntent.next_action by redirecting the user and then returning them to the checkout.

When this occurs, we shall require the three_d_secure_action_token_id be provided upon return, as follows:

const risk = recurly.Risk();
const threeDSecure = risk.ThreeDSecure({
  actionTokenId: myActionTokenId
});
threeDSecure.on('token', function (token) {
  // handle passing the action result
  // token back to your server

  // token.type => 'three_d_secure_action_result'
  // token.id

  // optionally, you may call threeDSecure.remove() to remove the element
});

threeDSecure.attach(document.querySelector('#my-auth-container'));

This change will handle re-submission of the action token, and determine if the Payment Intent has already been confirmed. If it is confirmed, then an action result token will be generated.

@chrissrogers chrissrogers force-pushed the stripe-use-handleNextAction branch from 1778bd4 to 849aa49 Compare December 4, 2024 20:37
@chrissrogers chrissrogers force-pushed the stripe-use-handleNextAction branch from 849aa49 to 3642335 Compare December 4, 2024 23:42
@btruncali1 btruncali1 merged commit 92727ae into master Dec 5, 2024
27 of 41 checks passed
@btruncali1 btruncali1 deleted the stripe-use-handleNextAction branch December 5, 2024 17:44
recurly-integrations pushed a commit that referenced this pull request Dec 10, 2024
[Full Changelog](v4.32.2...v4.32.3)

**Merged Pull Requests**

- Uses stripe.handleNextAction for payment intents [#908](#908) ([chrissrogers](https://github.com/chrissrogers))
- chore: add missing type definitions for Adyen APM [#907](#907) ([cbarton](https://github.com/cbarton))

**Closed Issues**

- Wrong type of AlternativePaymentMethods start and initiate [#885](#885)
- AlternativePaymentMethodOptions type is missing returnURL [#883](#883)
- AlternativePaymentMethods type missing iDEAL, Sofort and CashApp [#882](#882)
- Apple Pay wrongly uses the user's shipping country as billing country [#872](#872)

##### Minified MD5 Checksum
```
c357f439dd3d47115af8ed702ea2702e ./build/recurly.min.js
```
##### [SRI Hash](https://www.srihash.org/)
```
E3wPjw6EPYQZU6CWnkfKc+W76XcUWVZMwQNDVFHFk5uFvGKrVqa+5mk+f+AD62Ex
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants